Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • Streamlit 기본 pages/ 자동 로딩 방식을 제거하고 Navigation API 기반 구조로 변경
  • pages/ 디렉토리를 app_pages/로 이름 변경하여 중복 로딩 문제 방지
  • streamlit_app.py에서 st.Page, st.navigation()으로 페이지 구성을 명시적으로 정의
  • 전역 설정(page_title, page_icon 등) 추가하여 앱 기본 환경 개선

💬 To Reviewers (선택)

  • Navigation API 기반 구조로 변경한 것이 적절한지 확인 부탁드립니다.

PR Checklist

  • 멀티 페이지 구조 Navigation API 기반으로 리팩토링
  • pages/app_pages/ 디렉토리 이름 변경
  • streamlit_app.py에서 전역 설정 및 페이지 내비게이션 코드 반영
  • 로컬 환경에서 페이지 정상 동작 테스트 완료

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

@ParkGyeongTae ParkGyeongTae self-assigned this Oct 1, 2025
Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 너무 좋은데요!!! 일단 홈 화면 추가도 좋은 아이디어인것 같고 코드도 잘 구조화 된것같습니다.

image

💬 그래프 빌더쪽은 제가 추후에 좀 수정해서 어떤식으로 동작하는지 보는 페이지로 바꿔보던지 하겠습니댜ㅋㅋ

@ParkGyeongTae ParkGyeongTae merged commit 3636b89 into master Oct 3, 2025
2 checks passed
@ehddnr301 ehddnr301 deleted the 169-multipage-setup branch October 11, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants